Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[GH-253] Add filepath selection based on OS for nftables.conf #255

Merged
merged 4 commits into from
Sep 15, 2022
Merged

[GH-253] Add filepath selection based on OS for nftables.conf #255

merged 4 commits into from
Sep 15, 2022
+33 −5

Conversation

danielpkrueger
Copy link
Contributor

Description

Updates nftables recipe to support Oracle Linux 8 file paths.

Issues Resolved

[GH-253]

Check List

  • A summary of changes made is included in the CHANGELOG under ## Unreleased
  • New functionality includes testing.
  • New functionality has been documented in the README if applicable.

@danielpkrueger
Add nftables.conf location resource
75f9437 
-Added a resource to set the nftables.conf filepath based on OS platform.
-Updated inspec tests
@danielpkrueger
Update README.md
eddf34e
@danielpkrueger
Update CHANGELOG.md
88d5730
@danielpkrueger danielpkrueger requested a review from a team as a code owner September 8, 2022 20:51
@danielpkrueger danielpkrueger mentioned this pull request Sep 8, 2022
Open
@Stromweld Stromweld added the Release: Minor Release to Chef Supermarket as a minor release when merged label Sep 8, 2022
Stromweld
Stromweld requested changes Sep 8, 2022
View reviewed changes
resources/nftables.rb Outdated Show resolved Hide resolved
@Stromweld
Copy link
Contributor

@xorima, @ramereth would either of you know why the validators aren't running?

@xorima
Copy link
Contributor

xorima commented Sep 15, 2022

Fixed @Stromweld for now, as per slack I need to put in a long term fix

@Stromweld Stromweld merged commit 5cf59a0 into sous-chefs:main Sep 15, 2022
@kitchen-porter
Copy link
Contributor

Released as: 6.1.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bmhughes bmhughes bmhughes left review comments

@Stromweld Stromweld Stromweld approved these changes

Assignees
No one assigned
Labels
Release: Minor Release to Chef Supermarket as a minor release when merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@danielpkrueger @Stromweld @xorima @kitchen-porter @bmhughes